Skip to content

[Chore] Fix ci cat server log failed in cluster-test#17782

Merged
SbloodyS merged 1 commit intoapache:devfrom
ruanwenjun:dev_wenjun_fixCICatLog
Dec 19, 2025
Merged

[Chore] Fix ci cat server log failed in cluster-test#17782
SbloodyS merged 1 commit intoapache:devfrom
ruanwenjun:dev_wenjun_fixCICatLog

Conversation

@ruanwenjun
Copy link
Member

@ruanwenjun ruanwenjun commented Dec 12, 2025

Purpose of the pull request

When server is unhealthy, we will cat the server log, but the server log path is incorrect.

+ cat /home/runner/ds_test/worker-server/logs/dolphinscheduler-master.log
cat: /home/runner/ds_test/worker-server/logs/dolphinscheduler-master.log: No such file or directory

Brief change log

Change the log path when cat

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

@ruanwenjun ruanwenjun requested a review from SbloodyS as a code owner December 12, 2025 02:46
@github-actions github-actions bot added the CI&CD label Dec 12, 2025
@ruanwenjun ruanwenjun added this to the 3.4.0 milestone Dec 12, 2025
Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ruanwenjun ruanwenjun force-pushed the dev_wenjun_fixCICatLog branch from 4113594 to fa238f4 Compare December 19, 2025 08:05
@sonarqubecloud
Copy link

@SbloodyS SbloodyS merged commit 418f035 into apache:dev Dec 19, 2025
74 of 75 checks passed
@ruanwenjun ruanwenjun deleted the dev_wenjun_fixCICatLog branch December 22, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants